Skip to content

Conversation

xfq
Copy link
Member

@xfq xfq commented Jul 18, 2025

No description provided.

@xfq xfq requested review from plehegar, koalie and r12a July 18, 2025 02:30
</pre>

<p>Produces output in the page like this: <span class="codepoint" translate="no"><bdi lang="fr">é</bdi> [<code class="uname">U+00E9 LATIN SMALL LETTER E WITH ACUTE</code>]</span>.</p>
<p>Produces output in the page like this: <span class="codepoint" translate="no"><bdi lang="fr">é</bdi><code class="uname">U+00E9 LATIN SMALL LETTER E WITH ACUTE</code></span>.</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be at least a space character between the unicode character and its name. This would happen with a copy-paste of the code above because there's a CR (or LF, or CRLF) between, which HTML renderers will fold into a space character. If you want the sample code to be exactly the same as that which is used to "produce output in the page", just change my space to a CR/LF/CRLF.

Suggested change
<p>Produces output in the page like this: <span class="codepoint" translate="no"><bdi lang="fr">é</bdi><code class="uname">U+00E9 LATIN SMALL LETTER E WITH ACUTE</code></span>.</p>
<p>Produces output in the page like this: <span class="codepoint" translate="no"><bdi lang="fr">é</bdi> <code class="uname">U+00E9 LATIN SMALL LETTER E WITH ACUTE</code></span>.</p>

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@TallTed TallTed Jul 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ugh. As noted in that VERY LONG discussion, removing this space character is VERY counterintuitive. It's really unfortunate that there appears to be no clever way of treating an inline space character as if it weren't there for the CSS spacing. I do note that the template appears to have a line break following the </bdi> which gets folded into a space char which then increases the spacing due to the CSS. I won't fight it, but yeesh!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an LF in the <pre> block because we want the code to be clear, rather than having an extra long line. And there's no LF (or space) in the rendered result below.

@koalie koalie removed their request for review July 22, 2025 14:19
xfq added a commit to w3c/bp-i18n-specdev that referenced this pull request Jul 24, 2025
To reduce confusion. Also reduce the inline padding.

See w3c/guide#342 (comment)
@xfq
Copy link
Member Author

xfq commented Aug 7, 2025

I'll merge this, and we can discuss the space issue in w3c/bp-i18n-specdev#165

@xfq xfq merged commit 2b4e390 into main Aug 7, 2025
@xfq xfq deleted the char-naming-20250718 branch August 7, 2025 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants